Skip to content

Sprint improvements#29

Merged
irm-codebase merged 10 commits intomainfrom
fix-sprint-cleaning
Jul 28, 2025
Merged

Sprint improvements#29
irm-codebase merged 10 commits intomainfrom
fix-sprint-cleaning

Conversation

@irm-codebase
Copy link
Copy Markdown
Collaborator

Fixes #28

A collection of small to medium fixes discovered during the first modules sprint.

@irm-codebase irm-codebase self-assigned this Jul 21, 2025
@irm-codebase
Copy link
Copy Markdown
Collaborator Author

irm-codebase commented Jul 21, 2025

TODO:

@irm-codebase irm-codebase changed the title Sprint improvemetns. Sprint improvements Jul 21, 2025
@irm-codebase
Copy link
Copy Markdown
Collaborator Author

@sjpfenninger done with the general updates.

Most important bits are the removal of mkdocs (faster pixi builds) and running --cores 1 (no more windows failures, hopefully).

@irm-codebase
Copy link
Copy Markdown
Collaborator Author

irm-codebase commented Jul 25, 2025

More fixes:

  • add .pixi/ to gitignore
  • Add References file

Comment thread copier.yaml Outdated
Comment thread copier.yaml
- mkdocs.yaml
- pixi.toml
# Special cases
- "!config/README.md"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should README.md state explicitly that it is auto-generated and should not be modified?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure...
I did modify the README a bit in the geoboundaries and hydropower modules (an example image of the module results and a brief summary of steps). I guess these should go in the documentation instead... perhaps specify that these should have minimal changes to enable some flexibility?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok - I suggest to park this for now, revisit when finalising documentation

Comment thread template/workflow/Snakefile
@irm-codebase irm-codebase merged commit 0fb095f into main Jul 28, 2025
4 checks passed
@irm-codebase irm-codebase deleted the fix-sprint-cleaning branch July 28, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Post module sprint cleaning

2 participants